-
Notifications
You must be signed in to change notification settings - Fork 106
feat: barebones implementation of Blog Posts UI (WIP) #257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
bf0b43d to
5202e64
Compare
app/components/AppHeader.vue
Outdated
| <HeaderOrgsDropdown :username="npmUser" /> | ||
| </li> | ||
| <li class="flex items-center"> | ||
| <NuxtLink |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Kai-ros I'm wondering if we should move the blog link to the footer. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
I think nuxt prepare will try to compile standard-site-sync and since the lexicon types don't exist yet it will fail. We should generate the types first, and then run prepare.
6cebe53 to
7e87ee1
Compare
Also add some breathing room to the top of the blog post
|
Deployment failed with the following error: |
This is threadbare work, just basic bones of what's needed. Will add more tomorrow.